Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Extend VError support to include Error Cause proposal support #665

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

voxpelli
Copy link

See https://github.com/tc39/proposal-error-cause and tc39/proposal-error-cause#31 – a stage 3 proposal for including a VError style property to the JS language itself.

@hypesystem
Copy link

I was about to open a PR doing the same thing --- strong support from here 👍, we are currently extending the error serializer in all projects to support this.

@voxpelli
Copy link
Author

@hypesystem I would suggest switching to Pino if you can. Its pretty much equivalent + but more maintained + has eg this feature merged through pinojs/pino-std-serializers#78

@hypesystem
Copy link

I use pino for a bunch of projects, but the two aren't exactly equivalent, and there are plenty of situations where the cost of migrating is higher than the benefit 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants